Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added 2.0 version of the component along with Website project. #60

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Added 2.0 version of the component along with Website project. #60

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 5, 2018

-Library recompiled to .NetStandard 2.0
-Classes were moved to more appropriate namespaces:
* Constants
* Helpers
* Models
* ReportServiceExecution
-Renamed some classes with more meaningful names to avoid name conflicts.
-Improved language usage and removed unused references.

-Library recompiled to .NetStandard 2.0
-Classes were moved to more appropriate namespaces:
     Constants
     Helpers
     Models
     ReportServiceExecution
-Renamed nome classes with more meaningful names to avoid name conflicts
-Improved language usage and remived unused references
@Vincent95227
Copy link

Hello,
Your modifications can work with .net Core 2.0?
If Yes, How can I use it ?

Vincent.

@ghost
Copy link
Author

ghost commented Apr 6, 2018

Hello Vincent,
I forked Alan's repository and my changes can be found here. The component was recompiled as a separate project and It's compatible with .NET Core 2. I am currently using it on my project. Unfortunately it is not yet available as a Nuget package since Alan is the owner of the project. You would need to either download a copy and reference the project from your solution or compile it and reference the generated DLL.

I have additional changes that I haven't uploaded yet. The most remarkable one is the support for areas and the possibility to having multiple instances of the ReportController (with different names) which is not currently supported on the other version of the component.

If there is interest from the community, I can add these changes as well.

Carlos

-Improved responsive design and alignment of controls
@WW24865736
Copy link

Carlos,

I have used your version of the reportviewer for core 2.0.
It worked intermittently with an Area setup.
I had a controller called ReportController in Controllers folder.
I had same controller in Area called ServerApps.
So Report/MyReport and ServerApps/Report/MyReport would both work.
I found that removing the Reports controller from the typical controllers folder and leaving
the area one remaining, the area report failed to load.
Ideally I wanted to the area report as opposed to the typical controller.
Therefore, I would be very interested if you have fixed this scenario and you could post the solution.
If not, then I guess I will have to remove the area.

Regards,

Wayne

@ghost
Copy link
Author

ghost commented Aug 27, 2018

Hello Wayne,
I haven't looked at the issue you are describing. I am working on some other issues that I identified on the report component. I will take a look at this and I will update the repository as soon as I can, I need to meet a deadline on my current assignment.

Thanks for the input, Wayne,
Carlos

@nhaberl
Copy link

nhaberl commented Nov 20, 2019

Hey, have you plans to update also for .net core 3 ?

@sblasa
Copy link

sblasa commented Sep 18, 2021

Hello, has anyone tried using this with .NET Core 3.1 or higher?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants